Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introducing function.factor #885

Merged
merged 10 commits into from
Nov 1, 2024
Merged

Introducing function.factor #885

merged 10 commits into from
Nov 1, 2024

Conversation

gertjanvanzwieten
Copy link
Contributor

No description provided.

This patch modifies the hash used for the name of the function produced by
evaluable.compile, in order to reuse it as nutils hash.
This patch adds a simplification for LoopSum, that promotes inflations for
which the dofmaps are independent of index.
This patch removes the background colour of graphviz nodes that are not visited
once. For functions that are called more then once, this helps to distinguish
nodes with cached values from those that evaluate in a short time span.
@gertjanvanzwieten gertjanvanzwieten force-pushed the factor branch 2 times, most recently from 948761f to 8637fd0 Compare October 28, 2024 15:25
@gertjanvanzwieten gertjanvanzwieten marked this pull request as ready for review October 28, 2024 15:25
nutils/evaluable.py Outdated Show resolved Hide resolved
nutils/evaluable.py Outdated Show resolved Hide resolved
nutils/evaluable.py Show resolved Hide resolved
Gertjan van Zwieten and others added 7 commits November 1, 2024 16:17
This patch adds the argument_degree method to evaluable arrays, which returns
the highest power in a specific argument if the evaluable is polynomial in that
argument; or raises a NotPolynomial error otherwise.
This patch adds function.factor and evaluable.factor for the creation of
equivalent sparse polynomials. See the documentation of evaluable.factor for
details.
This patch replaces the trialshapes dictionary in System, which contains only
the shapes of the trial arguments, by the argshapes dictionary, which contains
all of the evaluable's arguments. The reason for the change is that the broader
set of arguments can be used for all sorts of argument checks prior to
initiating a solve.
This patch renames the System.step parameters timetarget to timearg and
historysuffix to suffix for brevity.
@gertjanvanzwieten gertjanvanzwieten merged commit f2a895b into master Nov 1, 2024
18 of 21 checks passed
@gertjanvanzwieten gertjanvanzwieten deleted the factor branch November 1, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants